@W-21741005: NEW - Implement Strategy Pattern for multi-engine custom rule creation#441
Open
aruntyagiTutu wants to merge 9 commits into
Open
@W-21741005: NEW - Implement Strategy Pattern for multi-engine custom rule creation#441aruntyagiTutu wants to merge 9 commits into
aruntyagiTutu wants to merge 9 commits into
Conversation
- Created strategy pattern infrastructure for extensible rule creation - Added IRuleCreationStrategy interface defining strategy contract - Implemented XPathRuleStrategy for PMD/XPath rules - Implemented RegexRuleStrategy for regex pattern rules - Created RuleCreationStrategyFactory for engine-based strategy selection - Refactored create_custom_rule tool to use strategy pattern - Tool now supports both engine: "pmd" and engine: "regex" - Added CreateRegexCustomRuleActionImpl for regex rule creation - Created temporary create_regex_rule tool for testing (to be deleted) - Updated provider to use RuleCreationStrategyFactory Benefits: - Single entry point for multiple engines - Easy to extend with new engines - Engine-specific logic isolated in strategies - Maintains backward compatibility with existing PMD functionality Regex rules support: - Inline YAML rules in code-analyzer.yml - Required: regex, violationMessage, tags, severity - Optional: fileExtensions, regexIgnore, includeMetadata - Validates regex format, severity range, file extensions
aruntyagiTutu
changed the title
- Refactored create_custom_rule.test.ts to use strategy mocks - Updated provider.test.ts to expect 7 tools (added temporary create_regex_rule) - All 228 tests passing - Coverage: 89.05% statements, 76.76% branches
…ests - Commented out create_regex_rule tool registration in provider - Tool code kept in codebase for reference only - Users can test regex rule creation via create_custom_rule with engine: 'regex' - Updated provider test to expect 6 tools instead of 7 - All 228 tests passing - E2E tests should now pass (9 tools expected)
- Added RegexRuleStrategy.test.ts with 26 tests - Added XPathRuleStrategy.test.ts with 20 tests - Added RuleCreationStrategyFactory.test.ts with 13 tests - Total: 59 new tests added - All 287 tests passing Coverage improvements: - RegexRuleStrategy: 8.69% → 100% - XPathRuleStrategy: 11.11% → 100% - RuleCreationStrategyFactory: 38.46% → 100% - Overall: 89.05% → 97.37% Test coverage includes: - Validation logic for all required/optional fields - Error handling for invalid inputs - Execute functionality for rule creation - File system operations (config creation/updates) - Factory pattern (strategy selection and registration) - Edge cases (case sensitivity, whitespace handling)
- Restored create_regex_rule.ts that was accidentally deleted - File kept for manual testing reference only (not registered in provider)
- Increased 'should enable 1 tool and a toolset' test timeout from 60s to 90s - Added connection timeout with Promise.race to fail fast on server issues - Connection timeout set to 90s for Windows compatibility This addresses E2E test failures on Windows runners where connection establishment takes longer than on Linux/macOS.
namrata111f
reviewed
Apr 17, 2026
namrata111f
reviewed
Apr 17, 2026
| "For Apex and Visualforce, use tool 'get_ast_nodes_to_generate_xpath' to generate the XPath." | ||
| ); | ||
| } else { | ||
| errors.push("xpath is required for PMD engine. Provide a valid XPath expression."); |
There was a problem hiding this comment.
Is LLM going to read these error messages and act on the basis of that?
namrata111f
reviewed
Apr 17, 2026
namrata111f
approved these changes
Apr 17, 2026
ZLeventer
reviewed
Apr 24, 2026
ZLeventer
left a comment
ZLeventer
left a comment
There was a problem hiding this comment.
The Strategy Pattern refactor for multi-engine rule creation is a meaningful architectural improvement. Having a pluggable engine interface makes it straightforward to add new analysis engines without modifying existing code paths.
From a user perspective, the key benefit here is extensibility — custom rule engines can be dropped in without touching the core MCP tool registration. This matters for teams that have proprietary static analysis rules they want to integrate alongside the built-in engines.
The workingDirectory parameter addition in #439 (which this builds on) is also important — without it, the MCP server assumes CWD which breaks in many IDE integration scenarios where the MCP server's CWD doesn't match the project being analyzed.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is to confirm that this is working in AFV.
@W-21741005@
Benefits:
Regex rules support:
Note: unit tests and testing is pending after stretegy pattern and need to delete create-regex-rule tool, that was created only for testing puprpose.